Skip to content

Conversation

@pahor167
Copy link
Contributor

@pahor167 pahor167 commented Aug 21, 2025

Adding a New Chain

This PR adds [Chain Name Here] to the registry.

Checklist

  • I have declared the chain at the appropriate Superchain Level.
  • I have run just codegen $SEPOLIA_RPC_URL,$MAINNET_RPC_URL to ensure that all auto-generated files are updated
  • I have checked Allow edits from maintainers on this PR.

@pahor167 pahor167 requested a review from a team as a code owner August 21, 2025 14:48
Copy link
Collaborator

@bitwiseguy bitwiseguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a few comments which should fix incorrect values and get the ci to pass (after your commit just codegen changes you create locally and push to this pr)

@@ -0,0 +1,66 @@
name = "Celo Sepolia"
public_rpc = ""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please populate this value, then re-run just codegen $SEPOLIA_RPC_URL to propagate the value into codegen'd files.

sequencer_rpc = "https://forno.celo-sepolia.celo-testnet.org"
explorer = "https://celo-sepolia.blockscout.com"
superchain_level = 2
governed_by_optimism = true
Copy link
Collaborator

@bitwiseguy bitwiseguy Aug 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Set this to false, then re-run just codegen $SEPOLIA_RPC_URL to propagate the value into codegen'd files.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove this file from the pr so it is not committed to the repo

@Wazabie Wazabie self-assigned this Sep 12, 2025
@Wazabie Wazabie added F-do-not-merge Flag: Do Not Merge M-new-chain-request Meta: New Chain Request labels Sep 12, 2025
@Wazabie
Copy link
Collaborator

Wazabie commented Sep 12, 2025

Hey @pahor167 I see unexpected changes in the files and you have ran just sync staging yourself while it was meant for us maintainers to run.

Can you please open a new PR where you just add the files in staging? Thank you!

@janjakubnanista janjakubnanista mentioned this pull request Oct 6, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

F-do-not-merge Flag: Do Not Merge M-new-chain-request Meta: New Chain Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants